Hue emulation module for Misterhouse #667
Conversation
Json server merge
…tion for mac addres
…e' when calling a configured sub so a real state can be returned by the sub
… to read_table_A.pl so users can define them the the mht file.
|
OK, Lieven, the CI stuff is great! It looks like there is a missing dependancy: IO::Socket::Multicast This brings up the whole site_libs vs loading separately, but I'm voting for keeping things encapsulated to make the install and setup as simple as possible. The Alexa stuff is a great compliment to homebridge, so IMHO it should work at startup. Wayne can you add in that dependancy and resubmit? |
|
I tried adding the module, but the test still fails. I am not sure whats going wrong with it. |
|
I’m just out this weekend, but I’ll download a fresh master and see if I can replicate the travis result. Do you want to send in a pull request against my master?
… On Feb 11, 2017, at 6:05 AM, Wayne Gatlin ***@***.***> wrote:
I tried adding the module, but the test still fails. I am not sure whats going wrong with it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#667 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AExbSLf9aVDWxTxoeFmMfU5Cu5pBkB3cks5rbbIxgaJpZM4L3V2I>.
|
|
I just created the pull request.
_Wayne
…On Sat, Feb 11, 2017 at 9:11 AM, hplato ***@***.***> wrote:
I’m just out this weekend, but I’ll download a fresh master and see if I
can replicate the travis result. Do you want to send in a pull request
against my master?
> On Feb 11, 2017, at 6:05 AM, Wayne Gatlin ***@***.***>
wrote:
>
> I tried adding the module, but the test still fails. I am not sure whats
going wrong with it.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub <
#667 (comment)>,
or mute the thread <https://github.com/notifications/unsubscribe-auth/
AExbSLf9aVDWxTxoeFmMfU5Cu5pBkB3cks5rbbIxgaJpZM4L3V2I>.
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#667 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFZ3efbaXi68NtzxsXaZ7Ld41qmB-IpCks5rbc-ugaJpZM4L3V2I>
.
|
|
Hey Wayne, Great! One more dependancy to add. CI dies on missing Net::Ethernet; Can't locate Net/Address/Ethernet.pm in @inc Can you add that dependancy and update the pull request? |
|
OK, one more (interesting, this one didn't show up when I installed IO::Socket::Multicast) Can't locate Net/Ifconfig/Wrapper.pm in @inc |
…g the local IP and can be manually defined in the ini.
|
Looks good, no errors at startup! |
|
Net::Address::Ethernet is only used for discovering the local IP and can be manually defined in the mh.private.ini so I just did a eval on it and log the error below if its not defined. This way the user can install the module if they want to. [Alexa] Error: Net::Address::Ethernet is not installed... Please install it or define the local IP and mac in alexaHttpIp/alexaMac |
This module has been tested with Google Home and the Amazon Echo. It should work for anything that supports the Hue bridge.